Skip to content

Fix relative links missing leading slash in Learn docs - #8568

Open
MyGO-Mujica wants to merge 1 commit into
reactjs:mainfrom
MyGO-Mujica:fix/learn-doc-links
Open

Fix relative links missing leading slash in Learn docs#8568
MyGO-Mujica wants to merge 1 commit into
reactjs:mainfrom
MyGO-Mujica:fix/learn-doc-links

Conversation

@MyGO-Mujica

Copy link
Copy Markdown

Summary

Fix 4 relative links in Learn docs that are missing the leading slash, causing them to resolve to incorrect 404 paths on the live site:

  • learn/preserving-and-resetting-state.md: learn/understanding-your-ui-as-a-tree#the-render-tree/learn/understanding-your-ui-as-a-tree#the-render-tree
  • learn/react-compiler/introduction.md: reference/react/useMemo#how-to-tell-if-a-calculation-is-expensive/reference/react/useMemo#how-to-tell-if-a-calculation-is-expensive
  • learn/setup.md: learn/react-developer-tools/learn/react-developer-tools
  • learn/understanding-your-ui-as-a-tree.md: learn/your-first-component#components-ui-building-blocks/learn/your-first-component#components-ui-building-blocks

Every other internal link in the repo uses absolute paths; these four are the only Learn docs links missing the leading slash.

Verification

  • Target pages and anchors verified to exist: understanding-your-ui-as-a-tree#the-render-tree, useMemo#how-to-tell-if-a-calculation-is-expensive, react-developer-tools, your-first-component#components-ui-building-blocks
  • Link scan across all src/content files confirms no other Learn links are affected

Four links in Learn docs were missing the leading slash and would resolve to 404 paths. Every other internal link in the repo uses absolute paths.
@github-actions

Copy link
Copy Markdown

Size changes

Details

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant